Repository navigation
feat!: delete kix, quest.moq.dev, and quest runner - #575
Conversation
Quest serves moq-dev only, and kix had never run on a real repository: its App, kixel-quest, was installed nowhere. moq-bot, which the host's daemons already act as, is the only GitHub App. Removed: the Worker (cloudflare/), `quest runner` (src/runner/) and its flake package and agent pins, the quest:kix hold label, kix's sessions comment that quest review and quest usage read, the Isolated and Sandbox workflows, the site-only theme assets, and the quests that built on kix. The land tests' fake GitHub moves to test/github.ts, cut to what they use. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
kixelated
left a comment
kixelated
left a comment
There was a problem hiding this comment.
Automated review by review (OpenAI)
Reviewed head 7827872d898a7d666e0a65d6ad9178b7334de5b0 against base 19c6602e4a137e96b974126c65b5e66e04def3c7.
No actionable correctness regressions found in the retained CLI changes. I checked the removal boundaries and remaining source/test imports, the single-label hold policy and its tests, PR-body-only model selection and usage attribution, the extracted Claude rate-limit reader against its previous implementation, the relocated land-test GitHub fake, and package/Nix/workflow changes. The rate-limit extraction preserves rejected-event, billing-error, and legacy reset handling.
Direction: the code is consistent with the stated removal scope; no blocking code change identified. The Worker, storage, DNS, and GitHub App cleanup listed in the PR remains a separate operator step, not something this diff performs.
Static GitHub-only review. I did not run builds, tests, Nix checks, or live CLI flows, and have not independently verified the author's reported test results or external deployment state. This is a comment, not a merge approval.
(Written by review (OpenAI))
Quest serves moq-dev only, so kix goes. It had never run on a real repository: its GitHub App, kixel-quest, was installed nowhere. moq-bot, which the host's daemons already act as, is now the only App.
What's removed
cloudflare/(the decision inbox, PWA, push notifications, boards, sign-in, and job routing at quest.moq.dev), along with its Isolated nightly workflow andjust worker-*recipes.quest runner(src/runner/), the flake'srunnerpackage and its pinned Claude Code, opencode, CLIProxyAPI, and sandbox-runtime, the Sandbox workflow,just runner-sandbox, and the@opencode/clientdependency.quest:kixhold label.quest:activeis the only hold, soacquire's check for a rival hold label went too.quest reviewnow reads the latest writer from the PR body's## Modelsrecord alone, andquest usagemakes one less API call per PR.theme.cssand the PWA icons).design/theme.mdkeeps the CLI's colours, the logo, and the social image.m0/kix,m1/kix, open, launch, user-state, theme,m2/cloud,m2/automation, donor-runner, donor-isolation, and mobile-app.m0,m1, andm2's goals drop kix, andm0's Plan records the decision. Two cosmetic CLI items from the theme epic are dropped with it: pluralisingquest check's "documents checked" and the column alignment for wide characters.The land tests' fake GitHub moves from
cloudflare/dev/github.tstotest/github.ts. It's cut from 1629 to 1135 lines by removing the paths that only the Worker exercised: SPDX, user tokens, and the inbox and job queries. The land tests reach the rest.Public API
Breaking:
quest runneris gone.packages.runner. No consumer used it: kixelated/nixos usespackages.default.quest/coreno longer exportsKIX_LABEL.quest:kixlabel no longer holds a PR.Nothing to migrate, since nothing used them.
Validation
just check,just test(1141 passed), andnix flake checkpass locally.After merge (operator)
Models
🤖 Generated with Claude Code
(Written by Claude Opus 5.5)
Review
Rewritten by
quest reviewafter each round.01a1bbff-992d-7952-8532-5a6269c5f45b, round 1The scoped diff consistently removes kix and its integrations, with no actionable regressions found by code inspection; builds and tests were not run, as instructed.